Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Digital Ocean Droplet Deployment Edit: Add info about nginx max file size to config instructions #2132

Merged

Conversation

abuhelos
Copy link
Contributor

What does it do?

Add another bullet + code block in "Install and configure Nginx web server" to mention that nginx has a "client_max_body_size" property that prevents uploads greater than 1MB by default, and to provide instructions on how to change it.

Why is it needed?

I was running into errors uploading media over 1MB to the admin panel and it took a bit to figure out that it was an Nginx problem.

Related issue(s)/PR(s)

strapi/strapi#9301

strapi/strapi#13743

@abuhelos abuhelos requested a review from pwizla as a code owner June 23, 2024 01:00
Copy link

vercel bot commented Jun 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2024 2:56pm

@strapi-cla
Copy link

strapi-cla commented Jun 23, 2024

CLA assistant check
All committers have signed the CLA.

@pwizla
Copy link
Collaborator

pwizla commented Jun 24, 2024

Thank you, @abuhelos!
LGTM. The PR is now approved, merged, should be live on docs.strapi.io in a few minutes, and will be mentioned in the next weekly snapshot release.

@pwizla pwizla added this to the 4.25.2 milestone Jun 24, 2024
@pwizla pwizla self-assigned this Jun 24, 2024
@pwizla pwizla added source: Dev Docs PRs/issues targeting the Developer Docs pr: updated content PRs updating existing documentation content labels Jun 24, 2024
@pwizla pwizla self-requested a review June 24, 2024 14:55
@pwizla pwizla merged commit 52ce924 into strapi:main Jun 24, 2024
2 of 3 checks passed
@pwizla pwizla added the temp: port to v5 Temporary label for PRs that need to be ported to v5 docs label Jun 24, 2024
@abuhelos abuhelos deleted the dev/edit-DO-droplet-deployment-instructions branch June 25, 2024 14:58
@pwizla pwizla removed the temp: port to v5 Temporary label for PRs that need to be ported to v5 docs label Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: updated content PRs updating existing documentation content source: Dev Docs PRs/issues targeting the Developer Docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Meta issue] 3rd-party deployment guides
3 participants